Skip to content

Archive fieldcompare diff files in diff-results/ on failure (fixes #441)#740

Open
PranjalManhgaye wants to merge 2 commits intoprecice:developfrom
PranjalManhgaye:issue-441-archive-fieldcompare-diffs
Open

Archive fieldcompare diff files in diff-results/ on failure (fixes #441)#740
PranjalManhgaye wants to merge 2 commits intoprecice:developfrom
PranjalManhgaye:issue-441-archive-fieldcompare-diffs

Conversation

@PranjalManhgaye
Copy link
Contributor

@PranjalManhgaye PranjalManhgaye commented Mar 4, 2026

Summary

Implements #441: archive fieldcompare diff VTK files so they are easy to find in CI artifacts when a comparison fails.

Changes

  • tools/tests/systemtests/Systemtest.py: When fieldcompare fails, copy *diff*.vtu, *diff*.vtk, and *diff*.vtp files from precice-exports/ into a diff-results/ folder in the same run directory.
  • tools/tests/README.md: Document that diff files are copied into diff-results/ for inspection (e.g. in ParaView) when debugging comparison failures.

Checklist

  • I added a summary of any user-facing changes (compared to the last release) in the changelog-entries/441.md.
  • I will remember to squash-and-merge, providing a useful summary of the changes of this PR.

@MakisH MakisH added GSoC Contributed in the context of the Google Summer of Code systemtests labels Mar 6, 2026
@PranjalManhgaye PranjalManhgaye changed the title Archive fieldcompare diff files in fieldcompare-diffs/ (fixes #441) Archive fieldcompare diff files in fieldcompare-diffs/ only on failure (fixes #441) Mar 7, 2026
@PranjalManhgaye PranjalManhgaye changed the title Archive fieldcompare diff files in fieldcompare-diffs/ only on failure (fixes #441) Archive fieldcompare diff files in diff-results/ on failure (fixes #441) Mar 7, 2026
@PranjalManhgaye PranjalManhgaye force-pushed the issue-441-archive-fieldcompare-diffs branch from 44fb3ea to a904f4f Compare March 7, 2026 07:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

GSoC Contributed in the context of the Google Summer of Code systemtests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants